Skip to content

Register #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 67 commits into
base: develop
Choose a base branch
from
Open

Register #6

wants to merge 67 commits into from

Conversation

El-Torador
Copy link
Collaborator

@El-Torador El-Torador commented May 29, 2020

Description

Please include a summary of the change and relevant motivation and context.
List any dependencies that are required for this change.

All pull requests should be related to open issues. Indicate the issue(s) bellow and remove this line:
Fixes #(issue number 1) #(issue number 2 if applicable)

How Has This Been Tested?

Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code etc.

Screenshots (if applicable, else remove this line / section)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added necessary inline code documentation
  • I have added tests that prove my fix is effective and that this feature works
  • New and existing unit tests pass locally with my changes

El-Torador added 30 commits May 16, 2020 04:30
Define PropTypes, props, handleSubmit method and displaying error message.
#1
Define the propTypes, props and pass it in the children components
#
Display the error message on UI
#done
Content the headers of the requests API and the API domain
#
El-Torador added 27 commits May 18, 2020 17:29
define the actions creators auth test
#
Pass even on param and reset form if request successfull
#
update Link,
pass all targets events in onsubmitForm
update toast msg struct
#
renme to ToastMessage and add type props, message and clearMessage
#
Create action creator of register feature and thunk function
#
add route to register feature
#
sign-up interfate for register user
#
the actions type for the regiter feature
#
register container to connect sign-up component with redux
#
Add reducer of register feature in combineReducers
#
Reducer for register feature
#
Add success props with props message inside and serialize Store
#
Copy link
Collaborator Author

@El-Torador El-Torador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

analysis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant